Skip to content

[16.0][REF] shopfloor_reception: block lot creation with a different expiration date#1168

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16.0-fix-shopfloor_reception-existing-lot-different-expiry-date
Jun 2, 2026
Merged

[16.0][REF] shopfloor_reception: block lot creation with a different expiration date#1168
OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16.0-fix-shopfloor_reception-existing-lot-different-expiry-date

Conversation

@nicolas-delbovier-acsone
Copy link
Copy Markdown

@nicolas-delbovier-acsone nicolas-delbovier-acsone commented May 29, 2026

Change the notification semantic when trying to scan or assign an existing lot with a different expiration date. Previously, this scenario raised a "warning" message type, which was misleading since the system blocks the operation as an error.

Update the message type from "warning" to "error" and simplify the message body.

Before After
image image

…tion date

Change the notification semantic when trying to scan or assign an existing
lot with a different expiration date. Previously, this scenario raised a
"warning" message type, which was misleading since the system blocks the
operation as an error.

Update the message type from "warning" to "error" and simplify the message body.
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @JuMiSanAr, @mmequignon,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added series:16.0 mod:shopfloor_reception Module shopfloor_reception labels May 29, 2026
@nicolas-delbovier-acsone
Copy link
Copy Markdown
Author

@jbaudoux

@jbaudoux
Copy link
Copy Markdown
Contributor

jbaudoux commented Jun 2, 2026

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-1168-by-jbaudoux-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2cbe50e into OCA:16.0 Jun 2, 2026
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 20860b0. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants